a11y: Avoid a warning

The listbox row accessible code was giving warnings when
parent is NULL, at destruction time.

GtkColorButton: Fix recursion loop in "use-alpha" set_property()

a11y: Disable deprecation warnings in AtkMisc implementation

combobox: Add workaround so evolution doesn't crash

Removing the NULL checks in 7f60cab47d9651ed3ed53b86f1f74de71b55eee0
caused evolution to crash. It chains up with NULL and that used to work.

This patch should be reverted for GTK 4.0.

https://bugzilla.gnome.org/show_bug.cgi?id=729496

applicationwindow: Properly unmap the menubar widget

tests: Add test for latest patch

This tesst infloops without it.

css: Fix printing code for CSS strings and idents

It used to infloop on special characters.

css: Split out a common function

It's used in two places already, and Iwant to use it in a third one.

Small documentation update

Don't show an example that doesn't work anymore.

Small documentation fixes

Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=729461

GtkApplication: Clarify docs a bit

Explicitly mention how gtk_application_set_accels_for_action can
be used to remove all accelerators for an action.

https://bugzilla.gnome.org/show_bug.cgi?id=729451

Updated Belarusian translation

listbox: check gsequence end iter in get_row_at_index

GtkHeaderBar: Allow overdrawing

The draw function of GtkHeaderBar was returning TRUE, causing
after-signal handlers to not run. This breaks e.g. drawing of
dnd hightlights. Fix that.

Fix a thinko in the last color button change

Using the interface method in the set_property implementation
leads to an infinite recursion, if the interface method is in
turn using g_object_set.

a11y tests: Avoid deprecation warnings

We can't drop the index-based AtkTable methods unless we change
the output format, so just ignore the deprecations for now.

a11y tests: Update expected output

The new AtkValue interface aways returns doubles, which causes
the range to be printed out with decimals.

a11y tests: Use the new AtkValue interface

The old interface is deprecated, and we just implemented the
new one, so lets test it.

GtkDialog: Reduce a g_warning to a g_message

Otherwise, this breaks our testsuite.

GtkSpinButton: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkScaleButton: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkRange: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkProgressBar: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkPaned: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkLevelBar: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkWidget: Mark deprecated properties as such

The ::style, ::margin-left and ::margin-right properties are all
deprecated.

Drop unneeded deprecation suppression

Removing this did not produce any deprecation warnings.

queryimmodules: Localize suppressed deprecation

Drop the blanket GDK_DISABLE_DEPRECATION_WARNINGS and avoid
deprecated API as much as possible.

GtkColorButton: Reduce internal use of deprecated API

Drop the blanket GDK_DISABLE_DEPRECATION_WARNINGS and avoid
deprecated API as much as possible.

GtkColorButton: Update deprecation warnings

It is not great to have deprecation warnings point to another
deprecated function. Update them to point to the eventual
replacement.

GtkDialog: Warn about lack of transient parents

We want make it mandatory for dialogs to have transient parents,
eventually. This is a first step in that direction.

teststack: don't enumerate transitions manually

Thats what introspection is for.

GtkStack: Avoid compiler warnings

gcc warns about unhandled enum values in these switch statements,
so add an empty default: case to placate the compiler.

stack: Add counterparts of OVER_UP_DOWN transition

The GtkStackTransitionType enum now has OVER_DOWN_UP, OVER_LEFT_RIGHT,
and OVER_RIGHT_LEFT values to complement OVER_UP_DOWN.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

stack: More readability when adding transitions

This small refactor makes the code more readable when adding more
transition types that have left, right, up, and down variations.

It adds inline boolean functions to tell information about transition
types (avoiding long if clauses) and changes long chains of "else if
(transition_type == ...)" into switch statements. Both are only likely
to get longer as more transition types are added.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

overlay: don't have a custom draw implementation

We can't think with Benjamin of a reason this should be needed, so let's
just remove it.

themingengine: only round outline corners after shrinking

Ensures the correctness of the applied radius.

GtkBuilder: added template parent type check.

GtkFlowBox: fixed NULL dereferencing segfaults

GtkMenu: fixed more NULL dereferencing segfault

GtkGrid: gtk_grid_get_size_for_size() do not dereference a NULL pointer.

GtkBuilder: improved parsing error report for invalid properties and signals.

Added GTK_BUILDER_ERROR_INVALID_PROPERTY and GTK_BUILDER_ERROR_INVALID_SIGNAL
error codes

ObjectInfo: Use a GType instead of a char * for the class name.
PropertyInfo: Use a GParamSpec instead of a char * for the property name.
SignalInfo: Use signal id and detail quark instead of a detailed signal name string.

This not only save us a few malloc in each case but lets us simplify the code
and report unknown properties and signals as a parsing error instead of just
printing a warning.

GtkBuilder: fixed a few minor memory leaks.

Fixed memory leaks in parse_object(), parse_template() and parse_signal() functions.
Parameters value where strduped before the last posible return and not freed.

W32: override selected button style to remove colored background

https://bugzilla.gnome.org/show_bug.cgi?id=729353

W32: Add a theme for .app-notification

This is copied from Adwaita. Looks sufficiently ok.

https://bugzilla.gnome.org/show_bug.cgi?id=729346

listbox: fix a segfault

Don't dereference a NULL pointer.

combobox: fix a segfault

Don't dereference a NULL pointer.

themingengine: use outlines to render focus rings

Instead of the old non-CSS implementation. This allows us to get rid of
the very last piece of custom theming in Adwaita.

css: add properties for outline-radius

Both a shorthand and individual properties. This is not officially part
of the CSS standard, but there's precedence for it at least in Mozilla:
https://developer.mozilla.org/en-US/docs/Web/CSS/-moz-outline-radius

treeview: respect focus-padding when rendering focus

Adwaita had a hack to work this around, but now that we're moving focus
ring rendering into GTK, we can just fix the treeview instead.

treeview: Always call size request funcs with valid out pointers

Not doing so causes crashes since
a158a2aa48c8023f99963642cc2657bff207b82NULLd

revealer: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

popover: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menuitem: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menu: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

levelbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

eventbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

listbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

frame: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

headerbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

infobar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

grid: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

scrolledwindow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

switch: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

toolbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

treeview: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Implement hfw properly

Previously, the code would fall back on GtkBin's hfw implementation
which didn't account for padding and borders.

viewport: Use a convenience function

That function looks longer than not using it, but we'll need it in the
next commit.

viewport: Request 2x border-width

It's nice that bugs get pointed out in code comments. Fixing them would
probably have been preferrable though.

widget: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

flowbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

combobox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

button: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

alignment: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

arrow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

spinner: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

modules: remove the pixbuf engine directory

This is not used anywhere, and is commented out of the build because it
doesn't compile. Time for some spring cleaning...

Updated POTFILES.skip

GtkSearchBar: fix the template

There was a mismatch between GtkSearchBar being derived from GtkBin
and the template claiming the parent to be GtkBox.

widget-factory: populate the second page more

Add back a menubar example that was lost a while ago, and also
include a searchbar and an infobar.

widget-factory: rearrange page 2 a bit

reftests: Add test for latest commit

Check that the label gets linebroken properly.

window: Subtract shadow from passed in width for height

... and height for width in size requests. Fixes mislayout in
control-center universal access panel.

widgets: Remove (GtkTickCallback) casts

Instead, make the functions conform to the prototype, so that casting
isn't needed.

GtkMenuSectionBox: Don't show separators for nested subsections

...unless they are labeled.

To implement this, keep track of the nesting depth of section boxes,
starting from 0 for the toplevel and the submenu ones, and only
insert unlabeled separators if the depth is at most 1.

GtkMenuSectionBox: Always show separators for labeled sections

...unless they are empty.

Add a comment with rules for separators

Added template test case to builder test.

GtkMenuSectionBox: don't leak

This wasn't fully fleshed out yet.

widget-factory: Add a horizontal-buttons example

Sadly, we don't have good icons for payment methods...

treeview: Fix prelight redraw when adjustment changes

When the adjustment changes (due to e.g. a mouse wheel scroll) we update
the prelight. The part that un-prelights the previous prelight was
broken by the the pixel cache, as it called update_prelight in the
middle of the scrolling operation, where the windows were moved
but the tree_view->priv->dy was not changed to the new value. This
caused the updates to the pixel cache to go to the wrong place.

We fix this by fully doing the scroll before we update_prelight().

https://bugzilla.gnome.org/show_bug.cgi?id=728284

3.13.1

Update output of a11y tests

These are all affected by the addition of the action_box in
GtkDialog or by the switch to popovers in GtkMenuButton.

GtkMenuShell: We want separators in submenus

This was probably an oversight when adding the merge_separators
parameter. Without this change, the testsuite/gtk/gtkmenu test
fails.

GtkMenuShell: Don't use a non-existing property

GtkMenuTrackerItem::visible was removed a few commits ago.
It is not necessary to bind visible anyway, since the menu
tracker will insert and remove items as their visibility
changes.

testmenubutton: Add a toggle for use-popover

gdkwindow: ignore shape on client-side windows when recomputing visible regions

Rendering doesn't do much about clipping drawing operations to the window shape,
although invalidation applies the shape to every window, leaving possibly trails
of "overrendered" content. So ensure the shape portions get invalidated too when
the window is moved/resized.

https://bugzilla.gnome.org/show_bug.cgi?id=729095

popover: Always make room for the tail on all sides on size request

Now that popovers may snap to any side with enough space, make enough
room on every side when requesting size, so that there's no w/h differences
at the time of setting the child allocation.

https://bugzilla.gnome.org/show_bug.cgi?id=729097

GtkRecentChooserMenu: Set current uri before activating an item.

When activating an item using mnemonics GtkRecentChooserMenu does not select a
item before calling the item-activated signal thus
gtk_recent_chooser_get_current_uri() always return the last selected item
instead of the activated one.

Fixes Bug 495105 "Open recent file keyboard shortcuts do not work correctly"